-
-
Notifications
You must be signed in to change notification settings - Fork 408
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix compilation error with recent ncurses versions #184
base: master
Are you sure you want to change the base?
Conversation
Does not work for me:
Also as you can see the |
Can you please build against https://github.com/drmpjz/cmatrix once more? This should fix the issue during configure (there was an extra q at the end of a line, need to get a new keyboard, the escape key doesn't always work :-( ) Not sure this will fix the addwstr error for you. My fix addresses the ncurses/ncursesw case, while from the output above it looks like you don't have a ncurses at all. But maybe I am confused, you never know :-) Can you please try a man addwstr from a terminal? Does it return something at all? If so it should list the relevant include file in the synopsis. Can you post this here please? |
kek here's the build log
I do have ncurses:
and
|
There is progress I think:
In the future, will just make listings suffice? |
The autoreconf issues are something you need to check on your end (you seem to be on a newer version of auto tools). Once we found a working link, please run autoupdate on your end to get rid of the warning (and I will check on my end if it still works with the older version of autotools. |
I am not sure what you mean by autoreconf issues and a working link. It's my first time working with autotools. I run autoupdate and here's what it changed:
No problem and thank you! |
Can you please try once more? And if you don't mind telling, what distro (and version) are you building on? |
I am running Gentoo 2.15 on amd64 with the profile Here's the build log:
|
LINES/COLS is no longer a variable in ncurses, but is implemented as functions returning the resp values. Therefore they can not be assigned values anymore.
Fix the code to use local variables instead.
Also fix problems with ncursesw (addwstr) and add logic for font location in openSUSE.